Skip to content

Conversation

@jordan-powers
Copy link
Contributor

Backports the following commits to 8.19:

…9169)

Follow-up to elastic#126492 to apply the json parsing optimization to strings
containing unicode characters and some backslash-escaped characters.

Supporting backslash-escaped strings is tricky as it requires modifying the
string. There are two types of modification: some just remove the backslash
(e.g. \", \\), and some replace the whole escape sequence with a new
character (e.g. \n, \r, \u00e5). In this implementation, the optimization
only supports the first case--removing the backslash. This is done by
making a copy of the data, skipping the backslash. It should still be more
optimized than full String decoding, but it won't be as fast as 
non-backslashed strings where we can directly reference the input bytes.

Relates to elastic#129072.
@jordan-powers jordan-powers added :Core/Infra/Core Core issues without another label >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Core/Infra Meta label for core/infra team labels Jun 12, 2025
@elasticsearchmachine elasticsearchmachine merged commit 8514c0d into elastic:8.19 Jun 12, 2025
16 checks passed
@jordan-powers jordan-powers deleted the backport/8.19/pr-129169 branch June 12, 2025 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Core/Infra/Core Core issues without another label >non-issue Team:Core/Infra Meta label for core/infra team v8.19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants